Repository navigation
improvement(naming): rename sandbox-tasks to sandboxes and standardize file names - #9044
Conversation
…r-name stutter in execution files
…nventions Renames every check:file-names baseline entry (stutter and redundant-suffix) so the baseline is empty, plus the same rule's uncaught cases: grandparent-domain prefixes in application/ and orchestration/ folders, loose stores moved to stores/<feature>/store.ts, non-hook files dropped from the use-* prefix, and application context/errors outliers.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
 [Medium impact] The PR appears safe to merge; no actionable issue was found. SummaryRenames
Reviews (1) · Last reviewed commit: "improvement(naming): resolve rename ambi..." · Reviewed by Greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
No issues found across 1411 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Note: This PR contains a large number of files. cubic selects up to 200 of the highest-priority eligible files for this review, so some files may not have been reviewed.
Re-trigger cubic
This branch was successfully deployed
1 participant
Summary
apps/sim/sandbox-tasks/toapps/sim/sandboxes/check:file-namesbaseline (stutter + redundant-suffix) so the baseline is now empty: executor handlers →handlers/<x>/handler.ts,uploads/contexts/<ctx>/<ctx>-file-manager.ts→file-manager.ts,utils/*-utils.ts→utils/*.ts,lib/auth/auth-client.ts→client.ts, etc.remote-sandbox/sandbox-spec.ts→spec.ts), grandparent-domain prefixes inapplication/andorchestration/folders (credentials/application/credential-crud.ts→crud.ts), andlib/execution/execution-signal.ts→signal.tsstores/<feature>/store.ts(fullscreen-origin,modals/variables,undo-redo/code,dashboards/cursor), droppeduse-from deploy helpers that export no hook, alignedapplication/outliers (catalog-context.ts→context.ts,error.ts→errors.ts)workflows/orchestration/workflow-lifecycle.ts,workflows/application/workflow-variables.ts,contexts/workspace/file-version-store.ts)vi.mockstrings, relative barrel re-exports, script baselines, rules/skills docs, and comments that named the old files. No logic changesType of Change
Testing
check:file-names(0 baselined),check:test-patterns,check:explicit-any,check:source-conventions, block-registry check, artifact generators (no drift)@/and relative specifier in changed files resolves to a fileChecklist
test-auditauthoring gate)